-
Notifications
You must be signed in to change notification settings - Fork 3.3k
fix(azure): add azure-anthropic support to router, evaluator, copilot, and tokenization #3158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…, and tokenization
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile OverviewGreptile Summary
Confidence Score: 3/5
Important Files Changed
Sequence DiagramsequenceDiagram
autonumber
participant UI as Workflow UI/Executor
participant RBH as RouterBlockHandler
participant EBH as EvaluatorBlockHandler
participant GR as /api/guardrails/validate
participant VH as validateHallucination()
participant PR as /api/providers
participant AZA as azure-anthropic provider
UI->>RBH: Execute Router block
RBH->>PR: POST providerRequest (providerId + creds)
PR-->>RBH: Provider response (route decision)
UI->>EBH: Execute Evaluator block
EBH->>PR: POST providerRequest (providerId + creds)
PR-->>EBH: Provider response (scores)
UI->>GR: POST guardrails_validate (input + model + providerCredentials)
GR->>VH: validateHallucination(input)
VH->>PR: executeProviderRequest(providerId, request)
PR->>AZA: If providerId=azure-anthropic, call Azure Anthropic SDK
AZA-->>PR: LLM response
PR-->>VH: content/tokens
VH-->>GR: score/reasoning
GR-->>UI: validation result
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
8 files reviewed, 2 comments
…anslate missing bedrockAccessKeyId
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
15 files reviewed, 3 comments
Additional Comments (2)
This needs to guard the spread (e.g.,
In Update the description to a provider-agnostic Azure endpoint (matching the new shared usage). |
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
15 files reviewed, 1 comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
- Pass all provider credentials unconditionally in router, evaluator (matching agent pattern) - Remove conditional if-branching on providerId for credential fields - Thread workspaceId through guardrails → hallucination validator for BYOK key resolution - Remove getApiKey() from hallucination validator, let executeProviderRequest handle it - Resolve vertex OAuth credentials in hallucination validator matching agent handler pattern Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
15 files reviewed, 1 comment
Additional Comments (1)
Fix by only attaching the credential fields for the active provider (as the code did before this PR), or by ensuring the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Summary
getProviderCredentialSubBlocks()so router and evaluator blocks show the Azure endpoint fieldazureEndpointin router and evaluator handlers for azure-anthropic providerType of Change
Testing
Tested manually
Checklist